The latest updates on your projects. Learn more about Vercel for Git ↗︎

I reviewed the pull request and for the most part the code looks good. Here are just a couple things I thought needed attention.

1. I don't think we should be uploading .DS_Store or at least I don't now what its adding.
2. If we are adding "stevensona.character-count" to devcontainer.json I don't think we should add extensions.json in our main repo.
3. The linear regression model uses possum.csv for there testing so I don't think we should remove it from data.
4. I tried to run the code but I don't have any test images so I got ValueError: No images found in directory data/. Allowed formats: ('.bmp', '.gif', '.jpeg', '.jpg', '.png'). So I think you should have some test images added to data so we could run it to get an idea of how it works.

I reviewed the pull request and for the most part the code looks good. Here are just a couple things I thought needed attention.

1. I don't think we should be uploading .DS_Store or at least I don't now what its adding.

2. If we are adding "stevensona.character-count" to devcontainer.json I don't think we should add extensions.json in our main repo.

3. The linear regression model uses possum.csv for there testing so I don't think we should remove it from data.

4. I tried to run the code but I don't have any test images so I got ValueError: No images found in directory data/. Allowed formats: ('.bmp', '.gif', '.jpeg', '.jpg', '.png'). So I think you should have some test images added to data so we could run it to get an idea of how it works.

I'll be able to address all of your concerns in the new pull request. Thanks for bringing up the issues it will really help me to focus on specific areas where changes are required.